Skip to content

fix(frontend): build on node:22-alpine so the lockfile's Node 22.13+ package's install - #1303

Merged
carddev81 merged 1 commit into
mainfrom
viteversionupdate-fix
Nov 11, 2026
Merged

carddev81 merged 1 commit into
mainfrom
viteversionupdate-fix

Conversation

@carddev81

Copy link
Copy Markdown
Contributor

build on node:22-alpine so the lockfile's Node 22.13+ packages install to avoid image build failures

@carddev81
carddev81 requested a review from a team as a code owner November 11, 2026 19:19
@carddev81
carddev81 requested review from CK-7vn and removed request for a team November 11, 2026 19:19
@coderabbitai

coderabbitai Bot commented Nov 11, 2026 •

Copy link
Copy Markdown

Explain Change →

No actionable comments were generated in the recent review.

Summary

Summary by CodeRabbit

  • Chores
    • Updated the frontend build and development environments to use the floating Node 22 Alpine image instead of the version pinned to Alpine 3.19. This changes the base image used when creating frontend containers; no user-facing features or behavior are changed.

Both frontend Dockerfiles now use node:22-alpine instead of node:22-alpine3.19.

Changes: Frontend image tags

Layer / File(s) Summary
Update frontend Dockerfile image tags
frontend/Dockerfile, frontend/dev.Dockerfile
The builder and development images now use node:22-alpine instead of node:22-alpine3.19.
Merge Risk: Minimal · up to 253f0

This change moves the frontend images to the floating Node 22 Alpine tag so packages that need Node 22.13 or later can install. No concrete merge-blocking risk remains.

Pre-merge checks: Passed 4

Check name Status Explanation
Title check Passed The title clearly identifies the main change: use node:22-alpine for frontend builds to support Node 22.13+ package installation and avoid build failures.
Description check Passed The description directly explains the frontend image change and its purpose of avoiding image build failures.
Linked Issues check Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

Share

Recent review info

Run configuration

  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: Organization UI
  • Review profile: ASSERTIVE
  • Plan: Advanced
  • Run ID: 2c806e6e-114f-4765-b8bc-56378c7e8a54
Commits

Reviewing files that changed from the base of the PR and between a732e09 and 253f0e6.

Files selected for processing: 2

  • frontend/Dockerfile
  • frontend/dev.Dockerfile

Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review.

@carddev81
carddev81 requested review from corypride and removed request for CK-7vn November 11, 2026 19:20
@carddev81
carddev81 merged commit 0671746 into main Nov 11, 2026
10 checks passed
@carddev81
carddev81 deleted the viteversionupdate-fix branch November 11, 2026 19:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant